Skip to content
This repository has been archived by the owner on Nov 18, 2024. It is now read-only.

Add font sizes #93

Merged
merged 11 commits into from
Aug 21, 2024
Merged

Add font sizes #93

merged 11 commits into from
Aug 21, 2024

Conversation

carolinan
Copy link
Contributor

@carolinan carolinan commented Aug 20, 2024

Description
A first go at adding the font sizes.
Closes #36

Screenshot
font-size-control-tt5

Testing Instructions

  1. Apply the PR
  2. Confirm that the sizes are applied correctly to the body, headings, and the updated blocks, for example
    Site title, tagline, search, code, post date.

Copy link

github-actions bot commented Aug 20, 2024

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: carolinan <[email protected]>
Co-authored-by: juanfra <[email protected]>
Co-authored-by: beafialho <[email protected]>
Co-authored-by: kafleg <[email protected]>
Co-authored-by: hanneslsm <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

theme.json Show resolved Hide resolved
theme.json Show resolved Hide resolved
@carolinan carolinan mentioned this pull request Aug 20, 2024
@kafleg
Copy link
Member

kafleg commented Aug 20, 2024

I checked this PR with @MichelleBulloch in a call and showed her how to test the PR. She is interested to learn and contribute. :)

Copy link
Member

@juanfra juanfra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks so much for working on this, Carolina! I left a few comments, most of them are related to the use of sizes in rem.

theme.json Outdated Show resolved Hide resolved
theme.json Outdated Show resolved Hide resolved
theme.json Show resolved Hide resolved
theme.json Outdated Show resolved Hide resolved
theme.json Outdated Show resolved Hide resolved
theme.json Show resolved Hide resolved
theme.json Outdated Show resolved Hide resolved
theme.json Outdated Show resolved Hide resolved
},
"h1": {
"typography": {
"fontSize": "70px"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to make this fluid?

},
"h2": {
"typography": {
"fontSize": "48px"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to make this fluid?

@carolinan carolinan added the [Status] Needs Design Feedback Needs general design feedback. label Aug 20, 2024
carolinan and others added 5 commits August 20, 2024 19:49
Co-authored-by: Juan Aldasoro <[email protected]>
Co-authored-by: Juan Aldasoro <[email protected]>
Co-authored-by: Juan Aldasoro <[email protected]>
Co-authored-by: Juan Aldasoro <[email protected]>
Co-authored-by: Juan Aldasoro <[email protected]>
theme.json Outdated Show resolved Hide resolved
Copy link
Member

@juanfra juanfra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Carolina! This looks good 🏅

Let's merge it and we can iterate from here, if there's anything to be adjusted.

@carolinan carolinan merged commit 146d0a7 into trunk Aug 21, 2024
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add font size presets
4 participants